Skip to content

dispatch kernel known event names#39

Merged
mmoreram merged 2 commits intodriftphp:masterfrom
skydiablo:master
Jan 27, 2021
Merged

dispatch kernel known event names#39
mmoreram merged 2 commits intodriftphp:masterfrom
skydiablo:master

Conversation

@skydiablo
Copy link
Copy Markdown
Contributor

No description provided.

@mmoreram
Copy link
Copy Markdown
Member

Hello @skydiablo

Thanks for the PR.
AFAIK, in the last Symfony kernel versions, the name is not necessary anymore. The same event namespace and name give this information.
Did you find an scenario where it is necessary?

Thanks.

@mmoreram
Copy link
Copy Markdown
Member

Oh, I saw this PR today!
I see the related PR.
Let me see this.

@mmoreram
Copy link
Copy Markdown
Member

Ok. If we want to use Symfony environment bundles, then we need to change these values.
Thanks for that addition!

@mmoreram mmoreram merged commit b03a085 into driftphp:master Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants